SonarSource Rules
  • Products

    In-IDE

    Code Quality and Security in your IDE with SonarQube Ide

    IDE extension that lets you fix coding issues before they exist!

    Discover SonarQube for IDE

    SaaS

    Code Quality and Security in the cloud with SonarQube Cloud

    Setup is effortless and analysis is automatic for most languages

    Discover SonarQube Cloud

    Self-Hosted

    Code Quality and Security Self-Hosted with SonarQube Server

    Fast, accurate analysis; enterprise scalability

    Discover SonarQube Server
  • SecretsSecrets
  • ABAPABAP
  • AnsibleAnsible
  • ApexApex
  • AzureResourceManagerAzureResourceManager
  • CC
  • C#C#
  • C++C++
  • CloudFormationCloudFormation
  • COBOLCOBOL
  • CSSCSS
  • DartDart
  • DockerDocker
  • FlexFlex
  • GitHub ActionsGitHub Actions
  • GoGo
  • HTMLHTML
  • JavaJava
  • JavaScriptJavaScript
  • JSONJSON
  • JCLJCL
  • KotlinKotlin
  • KubernetesKubernetes
  • Objective CObjective C
  • PHPPHP
  • PL/IPL/I
  • PL/SQLPL/SQL
  • PythonPython
  • RPGRPG
  • RubyRuby
  • RustRust
  • ScalaScala
  • ShellShell
  • SwiftSwift
  • TerraformTerraform
  • TextText
  • TypeScriptTypeScript
  • T-SQLT-SQL
  • VB.NETVB.NET
  • VB6VB6
  • XMLXML
  • YAMLYAML
Go

Go static code analysis

Unique rules to find Bugs, Vulnerabilities, Security Hotspots, and Code Smells in your GO code

  • All rules 94
  • Vulnerability21
  • Bug13
  • Security Hotspot14
  • Code Smell46
 
Tags
    Impact
      Clean code attribute
        1. Context should not be stored in struct fields

           Code Smell
        2. Context parameters should be reused instead of creating new background contexts

           Code Smell
        3. Package imports should be consistent and avoid redundancy

           Code Smell
        4. Variables should be used

           Code Smell
        5. Consecutive function parameters with the same type should be grouped

           Code Smell
        6. Named types should be used instead of anonymous structs for complex nested structures

           Code Smell
        7. Use "bytes.Equal" instead of "bytes.Compare" for equality checks

           Code Smell
        8. Single-method interface names should follow Go naming conventions

           Code Smell
        9. Variables in if short statements should be used beyond just the condition

           Code Smell
        10. Context cancellation functions should be deferred

           Code Smell
        11. Blank imports should be documented to explain their purpose

           Code Smell
        12. Function and method names should not use "Get" prefix

           Code Smell
        13. Semicolons should not be used unnecessarily

           Code Smell
        14. Import statements should be factored into a single block

           Code Smell
        15. Functions should follow Go's explicit error handling patterns

           Code Smell
        16. Multi-line comments should not be empty

           Code Smell
        17. Functions should not have identical implementations

           Code Smell
        18. Cognitive Complexity of functions should not be too high

           Code Smell
        19. Go parser failure

           Code Smell
        20. Boolean checks should not be inverted

           Code Smell
        21. Two branches in a conditional structure should not have exactly the same implementation

           Code Smell
        22. "switch" statements should not be nested

           Code Smell
        23. "switch" statements should not have too many "case" clauses

           Code Smell
        24. Track lack of copyright and license headers

           Code Smell
        25. Functions and methods should not have too many lines

           Code Smell
        26. Control flow statements "if", "for" and "switch" should not be nested too deeply

           Code Smell
        27. Octal values should not be used

           Code Smell
        28. "switch" statements should have "default" clauses

           Code Smell
        29. "if ... else if" constructs should end with "else" clauses

           Code Smell
        30. Statements should be on separate lines

           Code Smell
        31. String literals should not be duplicated

           Code Smell
        32. Functions should not be empty

           Code Smell
        33. Unused function parameters should be removed

           Code Smell
        34. Local variable and function parameter names should comply with a naming convention

           Code Smell
        35. "switch case" clauses should not have too many lines

           Code Smell
        36. Track uses of "TODO" tags

           Code Smell
        37. Track uses of "FIXME" tags

           Code Smell
        38. Boolean literals should not be redundant

           Code Smell
        39. Empty statements should be removed

           Code Smell
        40. Redundant pairs of parentheses should be removed

           Code Smell
        41. Nested blocks of code should not be left empty

           Code Smell
        42. Functions should not have too many parameters

           Code Smell
        43. Expressions should not be too complex

           Code Smell
        44. Files should not have too many lines of code

           Code Smell
        45. Lines should not be too long

           Code Smell
        46. Function names should comply with a naming convention

           Code Smell

        Function and method names should not use "Get" prefix

        consistency - conventional
        maintainability
        Code Smell
        • convention
        • naming

        This is an issue when function or method names start with 'Get' or 'get' prefix, unless the underlying concept specifically uses the word 'get' (like HTTP GET operations).

        Why is this an issue?

        How can I fix it?

        More Info

        In Go, using 'Get' prefixes in function and method names is considered redundant and goes against established naming conventions. The Go community follows the principle that functions returning something should be given noun-like names, starting directly with the noun rather than prefixing it with 'Get'.

        This convention exists because the function signature already makes it clear that something is being returned. Adding 'Get' creates unnecessary verbosity without adding meaningful information. For example, func (c *Config) GetJobName() is less idiomatic than func (c *Config) JobName().

        The 'Get' prefix pattern comes from other programming languages like Java, where getters and setters are common. However, Go’s philosophy emphasizes simplicity and conciseness, making such prefixes unnecessary.

        Following this convention makes code more readable and consistent with the broader Go ecosystem, including the standard library and most open-source Go projects.

        What is the potential impact?

        Using 'Get' prefixes in function names reduces code readability and makes the codebase inconsistent with Go naming conventions. While this doesn’t cause functional issues, it can:

        • Make the code appear less idiomatic to Go developers
        • Reduce maintainability as the naming doesn’t follow community standards
        • Create inconsistency when working with standard library and third-party packages that follow Go conventions
        • Make function names unnecessarily verbose
          Available In:
        • SonarQube CloudDetect issues in your GitHub, Azure DevOps Services, Bitbucket Cloud, GitLab repositories

        © 2025 SonarSource Sàrl. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use